Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add nucleotide-count exercise #31

Merged
merged 5 commits into from
May 27, 2024
Merged

Add nucleotide-count exercise #31

merged 5 commits into from
May 27, 2024
+127 −0

Conversation

ErikSchierboom
Copy link
Member

No description provided.

@ErikSchierboom ErikSchierboom mentioned this pull request May 26, 2024
Open
69 tasks
@ErikSchierboom ErikSchierboom marked this pull request as ready for review May 26, 2024 12:07
@BNAndras BNAndras added x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation labels May 26, 2024
@BNAndras BNAndras force-pushed the nucleotide-count branch 2 times, most recently from 22d6f88 to f073cee Compare May 26, 2024 18:29
@BNAndras
Copy link
Member

It seems = won't work when comparing the dictionaries. equal? works but it returns false for the dictionaries. In the REPL, it looks okay. I suspect there might be a big in unitt so I'll mention it on Discord later.

@BNAndras BNAndras force-pushed the nucleotide-count branch from ead8c23 to 8bb2527 Compare May 27, 2024 20:55
@BNAndras BNAndras merged commit 14906b0 into main May 27, 2024
2 checks passed
@BNAndras BNAndras deleted the nucleotide-count branch May 27, 2024 20:56
@ErikSchierboom
Copy link
Member Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BNAndras BNAndras BNAndras approved these changes

Assignees
No one assigned
Labels
x:module/practice-exercise Work on Practice Exercises x:rep/large Large amount of reputation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ErikSchierboom @BNAndras